RDKEMW-19163 thunder-migration-License - #521
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates STBService Thunder implementation source headers to reflect current licensing metadata and improve generated documentation via Doxygen file headers.
Changes:
- Updated copyright header year in multiple
*_Thunder.cppcomponent implementations. - Added Doxygen
@file/@brieffile-level comments to several Thunder-backed STBService sources.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/hostif/profiles/STBService/Components_VideoOutput_Thunder.cpp | Updates header copyright year; adds Doxygen file header. |
| src/hostif/profiles/STBService/Components_VideoDecoder_Thunder.cpp | Updates header copyright year; adds Doxygen file header. |
| src/hostif/profiles/STBService/Components_SPDIF_Thunder.cpp | Updates header copyright year (and file header present in current version). |
| src/hostif/profiles/STBService/Components_HDMI_Thunder.cpp | Updates header copyright year; adds Doxygen file header. |
| src/hostif/profiles/STBService/Components_DisplayDevice_Thunder.cpp | Updates header copyright year; adds Doxygen file header. |
| src/hostif/profiles/STBService/Components_AudioOutput_Thunder.cpp | Updates header copyright year (and file header present in current version). |
| src/hostif/profiles/STBService/Capabilities_Thunder.cpp | Adds Doxygen file header. |
Code Coverage Summary |
|
Hi @santoshcomcast : The BD/Copyright checks are passing so compliance is not holding up merge. As a general rule, we recommend the earliest year (i.e. file creation) as the copyright year in a file and in NOTICE. For existing files, it would be best to leave them alone. For newly created files, use the current year. Using a range, they will also go out of date in any case. However, we don't enforce particular years. |
nhanasi
approved these changes
Aug 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
update license year and dioxygen comment